-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Weapon drop callback + Fix for weapons still dropping #692
Conversation
okay so I tested the code callback now and the flag, and both work havent tested in specifically Infection though |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question and some formatting
Northstar.CustomServers/mod/scripts/vscripts/mp/_codecallbacks.gnut
Outdated
Show resolved
Hide resolved
Co-authored-by: Jack <[email protected]>
…hstarMods into weapon-drop-callback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, haven't tested yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looked at: everything (did not test, only looked at code)
code is fine, diffs are hard to read but git hates you ig.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting stuff
Northstar.CustomServers/mod/scripts/vscripts/mp/_codecallbacks.gnut
Outdated
Show resolved
Hide resolved
Northstar.CustomServers/mod/scripts/vscripts/mp/_codecallbacks.gnut
Outdated
Show resolved
Hide resolved
Co-authored-by: Jack <[email protected]>
Co-authored-by: Jack <[email protected]>
Co-authored-by: Jack <[email protected]>
Co-authored-by: Jack <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updating review because for some reason its still blocking?
Anyway yeah code looks good
Tested on infection with latest northstar version. Can confirm that this works as intended, executions no longer bypass the weapon drop flag. |
As of making this PR its untested.
This adds a callback for when a gun gets dropped, this is just an implementation of the already existing CodeCallback.
It at the same time fixes #613 and is a better #617 , it removed the callback for deleting the dropped gun.
Closes #613